Skip to content

CI: Reduce redundant Rust checks - #2703

Merged
LucaCappelletti94 merged 1 commit into
mainfrom
ci-reduce-rust-check-overlap
Oct 23, 2026
Merged

CI: Reduce redundant Rust checks#2703
LucaCappelletti94 merged 1 commit into
mainfrom
ci-reduce-rust-check-overlap

Conversation

@LucaCappelletti94

@LucaCappelletti94 LucaCappelletti94 commented Oct 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

CI uses separate runner slots for compiler checks already covered by clippy. Workspace-wide linting covers the parser and benchmark targets, while line-tables-only retains file-and-line backtraces for test failures.

cc @alamb will merge this one when possible as it slims down the CI, but given we are still backlogged I am not sure what is the current cause.

@LucaCappelletti94 LucaCappelletti94 mentioned this pull request Oct 21, 2026
Merged
@LucaCappelletti94
LucaCappelletti94 changed the base branch from main to ci-drop-redundant-required-checks October 21, 2026 12:04
@LucaCappelletti94
LucaCappelletti94 force-pushed the ci-reduce-rust-check-overlap branch from 45274ed to 3ccdceb Compare October 21, 2026 12:04
@LucaCappelletti94
LucaCappelletti94 added this pull request to stack #2705 October 21, 2026 12:11
@alamb

alamb commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

: cc @alamb will merge this one when possible as it slims down the CI, but given we are still backlogged I am not sure what is the current cause.

I think the CI usage in this project is a drop in the bucket compared to other projects that share the same runner. Thus I am not sure how much effort it is worth trying to optimize our CI here much more

@LucaCappelletti94

LucaCappelletti94 commented Oct 21, 2026

Copy link
Copy Markdown
Contributor Author

I think the CI usage in this project is a drop in the bucket compared to other projects that share the same runner. Thus I am not sure how much effort it is worth trying to optimize our CI here much more

Yeah I did this PR before understanding the size of the problem.

@codecov-commenter

codecov-commenter commented Oct 21, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.53%. Comparing base (9119ec6) to head (63ff82c).

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #2703   +/-   ##
=======================================
  Coverage   81.53%   81.53%           
=======================================
  Files          42       42           
  Lines       34218    34218           
  Branches    34218    34218           
=======================================
  Hits        27898    27898           
  Misses       2770     2770           
  Partials     3550     3550

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Base automatically changed from ci-drop-redundant-required-checks to main October 23, 2026 10:37
@LucaCappelletti94
LucaCappelletti94 force-pushed the ci-reduce-rust-check-overlap branch from 3ccdceb to 63ff82c Compare October 23, 2026 10:37
@LucaCappelletti94
LucaCappelletti94 enabled stacked auto-merge October 23, 2026 10:55
@LucaCappelletti94
LucaCappelletti94 added this pull request to the merge queue Oct 23, 2026
Merged via the queue into main with commit 383cc0b Oct 23, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alamb alamb alamb approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@LucaCappelletti94 @alamb @codecov-commenter